Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Optimized ScaleonPress widget #5

Open
wants to merge 57 commits into
base: master
Choose a base branch
from
Open

Optimized ScaleonPress widget #5

wants to merge 57 commits into from

Conversation

erluxman
Copy link
Owner

No description provided.

erluxman and others added 30 commits June 22, 2020 18:00
@erluxman
Merge pull request #1 from erluxman/splash_screen
41811d7 
Splash Screen Feature
@erluxman
Add AnimatedScale Widget
554d6ef
@erluxman
Improve the UI of Splash Screen
d6ad8da
@erluxman
added tests
24f48c5
@erluxman
compelted designing the bottom navbar
083aacf
@erluxman
Remove the leading < button on appbar of home
44ea612
@erluxman
removed debug tag for animated scale
a0faa2d
@erluxman
removed an unused import
9fe67b3
@erluxman
Improved test for MaterialApp
5bd5129
@erluxman
Add sound effects on splashscreen
a79ccff
@erluxman
some animation
c9fe80f
@erluxman
Bottom nav bar icons animation done
dd3da7d
@erluxman
removed unused imprt
544c53d
@erluxman
Replaced hardcoded stings and asset path with resrouce class
ffc4698
@erluxman
Fixed splash test
7101e9c
@erluxman
Completed Refactoring
0084bb3
@erluxman
Wrapped Splashcreen around Material App in testing
5c0e2cc
@erluxman
Added test coverage for Fab
11bde28
@erluxman
Added code coverage for Appbar
bd975ba
@erluxman
Refactored Bottomnav icon
5ce4a22
@erluxman
Removed unused import
40bb54f
@erluxman
added some sandbox navigation testing code
2ef95ed
@erluxman
Oh ye, the issue is not with splash screen, fully covered
9af98b1
@erluxman
Add build modes and make UI during testing if they make network call
7fe7837
@erluxman
Added preview dependency
fa60bcb
@erluxman
Added test for main() method
e88ce90
@erluxman
Removed navigation to app from playground
8545c68
@erluxman
Optimized imports
36abfbc
@erluxman
Added test for gradients
685c6be
@erluxman
Removed some flaky tests
2af773f
@codecov
Copy link

codecov bot commented Jul 24, 2020 •
edited
Loading

Codecov Report

Merging #5 into master will increase coverage by 8.87%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   89.76%   98.64%   +8.87%     
==========================================
  Files          15       15              
  Lines         303      295       -8     
==========================================
+ Hits          272      291      +19     
+ Misses         31        4      -27
Impacted Files Coverage Δ
lib/features/home/widgets/home_fab.dart 72.72% <ø> (ø)
lib/features/home/widgets/home_screen.dart 100.00% <ø> (+25.00%) ⬆️
lib/features/home/widgets/home_bottom_nav_bar.dart 97.05% <50.00%> (+30.39%) ⬆️
lib/features/splash/splash_screen.dart 100.00% <100.00%> (ø)
lib/utils/animation/scale_on_press_widget.dart 100.00% <100.00%> (+2.94%) ⬆️
lib/features/home/widgets/home_app_bar.dart 100.00% <0.00%> (+10.00%) ⬆️
lib/features/home/widgets/home_body.dart 100.00% <0.00%> (+25.00%) ⬆️
lib/features/home/widgets/animated_nav_button.dart 100.00% <0.00%> (+29.72%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 49707e0...a7e731e. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@erluxman